Remove GDestroyNotify casts in g_clear_pointer() uses

GLib master propagates argument types in g_clear_pointer(), which causes
the usual function pointer casts to GDestroyNotify to trip compiler
warnings. Additionally, this commit changes some cleanup functions where
appropriate (wl_data_source_destroy ->
gtk_primary_selection_source_destroy for struct
gtk_primary_selection_source).

render: Make gtk_render_icon() use the snapshot API

This removes a lot of duplicated code.

render: Make gtk_render_frame() and gtk_render_focus() use the snapshot API

This removes a lot of duplicated code.

API: Remove gtk_render_frame_gap()

That function does not make sense at all in a CSS world. So better don't
support it anymore.

render: Make gtk_render_background() use the snapshot API

This removes a lot of duplicated code.

render: Make gtk_render_check/arrow/handle() use the snapshot API

This removes a lot of duplicated code.

render: Make gtk_render_layout() use the snapshot API

This removes a lot of duplicated code.

testsuite: Fix DragContext => Drag renaming

gtk: Move GtkGesture declaration to gtktypes.h

That way, headers can use the gesture type without having to include
gtkgesture.h

docs: Remove remains of Mir backend

broadway: Remove unused variable

Update Romanian translation

Merge branch 'nirbheek/misc-meson-fixes' into 'master'

Fix broadway backend build and graphene as a subproject

Closes #1197 and #1218

See merge request GNOME/gtk!265

Updated Spanish translation

Merge branch 'just-popover-menu-things' into 'master'

Post-gizmo GtkPopoverMenu fixes

See merge request GNOME/gtk!264

meson: Update pango and gdk-pixbuf wrap locations

See: https://gitlab.gnome.org/GNOME/gtk/issues/1219

meson: Support graphene as a fallback dependency

Don't assume graphene_dep is always a pkg-config dependency.

Closes https://gitlab.gnome.org/GNOME/gtk/issues/1197

meson: libgdk-broadway.a depends on broadwayjs.h

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/1218

Also fix another dependency error I found.

Updated Spanish translation

testpopover: Don’t set margin in menu box

The gizmo already provides sufficient padding.

gtkpopovermenu: Conditionally chain up in gtk_popover_menu_add()

Otherwise the stack gets parented to the wrong widget and the contents
are never drawn.

gtkpopovermenu: Drop call to gtk_widget_show()

The stack is already visible.

Drop builtin icon test

We don't have these icons anymore.

Update British English translation

Merge branch 'wip/muktupavels/remove-gtk-css-provider-get-default' into 'master'

gtkcssprovider: remove gtk_css_provider_get_default

See merge request GNOME/gtk!256

ci: add hicolor-icon-theme to the docker image

The tests complain about it, not sure if this helps.

Drop unused old icons

We don't need to ship the deprecated old stock-id named
icons and the drag cursors - we don't use them from resources
anyway.

Update Turkish translation

Emoji chooser: Also check for hex boxes

We don't want to see those either.

moji chooser: Try harder to avoid fallback

We don't want to see any fallback rendering.
The current check was still letting some fallback
combinations through.

Based on work by Julian Sparber.

Quiet a compiler warning

We don't need to handle GDK_CONFIGURE here, so make the
compiler not warn about it.

testsuite: Add emoji icon names

We check for all icon names that are used in gtk.

Emoji chooser: Use icons for sections

This is more reliable than using text-style Emoji
which may not be present.

Based on work by Julian Sparber.

Merge branch 'master' into 'master'

widgetfactory: add separator tool item to presentation

See merge request GNOME/gtk!249

Merge branch 'content-provider-autoptr' into 'master'

gdk: Add auto cleanup func for GdkContentProvider

See merge request GNOME/gtk!251

Merge branch 'content-provider-docs' into 'master'

docs: Fix GdkContentProvider documentation

See merge request GNOME/gtk!254

docs: Fix GdkContentProvider documentation

Currently it’s lacking properties, signals and class vfuncs.

gtkcssprovider: remove gtk_css_provider_get_default

This function is unused since cdc6e8272020 commit and does not
contain fallback style.

https://gitlab.gnome.org/GNOME/gtk/issues/1226

Merge branch 'content-deserializer-fixes' into 'master'

Content deserializer fixes

See merge request GNOME/gtk!252

gdk: contentdeserializer: Fix deserializer not returning

file_uri_deserializer does not return on success, only on failure. This
commit fixes clipboard reads never finishing for some types.

gdk: contentdeserializer: Fix stream usage

file_uri_deserializer splices a memory stream, as opposed to
string_deserializer, which uses a converter and filter stream. This
commit fixes erroneous use of GMemoryOutputStream as
GFilterOutputStream.

gdk: Add auto cleanup func for GdkContentProvider

docs: Small updates to the migration guide

Mention that event controllers are available in 3.x, amongst others.

Merge branch 'places-sidebar-dnd' into 'master'

placessidebar: Take a GdkDrag in _set_drop_targets_visible()

Closes #1220

See merge request GNOME/gtk!247

widgetfactory: add separator tool item to presentation

placessidebar: Take a GdkDrag in _set_drop_targets_visible()

Since the function is usually called from GtkWidget::drag-{begin,end} handlers,
taking a GdkDrop does not work, especially given that
::drag-action-requested is emitted without checking the type.

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/1220

gesturestylus: Clarify a documentation comment

Use the proper gtkdoc syntax for signals.

Remove icon extents API

widget: Remove another ->priv usage

sizerequest: Pull locals into closest scope

Correctly mark an enum value in a doc comment

sizerequest: Only check reported baselines if requested

If the passed-in store locations for the baselines are NULL anyway, we
don't need to check the reported baselines for correctness.

gl renderer: Pull a few declarations into the closest scope

Remove GTK_CSS_AFFECTS_CLIP

Clips don't exist anymore and this flag is unused outside of the style
property definitions in gtkcssstylepropertyimpl.c

window: Don't queue an allocate if the css clip changes

clips don't exist anymore.

GskRoundedRect: Typo

gl renderer: Use a GArray for the shadow cache

It's very small usually, in default Adwaita the only blurred outset
shadow we have is the one for the CSD'd toplevel window.

Avoid a g_object_get path

We can use gdk_device_get_n_axes instead and will get the same result.

docs: Fix scroll and motion controller titles

Controller, not Controler.

Merge branch 'legacy-gl' into 'master'

fix the OpenGL renderer for legacy contexts

See merge request GNOME/gtk!240

Update POTFILES.in

Handle configure events in gdk

The previous attempt at removing configure events entirely
was causing some dialogs not to show up under Wayland.
Presumably due to ordering issues with emitting ::size-change
out of the backend.

Instead, keep configure events in the event queue, but handle
them on the gdk side. This keeps the ordering intact, while
still removing configure events from the api. The dialogs
show up now.

Revert "gdk: Drop configure events"

This reverts commit a8926c9d873ce968353a2eb1d3930c4f1ac79c94.

Revert "broadway: Stop using configure events"

This reverts commit 9c827cbff4d7cd3e59fbfeb7ec5abe9c7dbf6bdb.

Merge branch 'dnd-cleanups' into 'master'

Dnd cleanups

See merge request GNOME/gtk!243

win32: Stop accessing GdkDrag members

wayland: Set actions on drag

We should pass the actions to the newly created
drag in drag_begin.

x11: Set actions on drag

We should pass the actions to the newly created
drag object in drag_begin.

x11: Rename a file

Keep in line with the parent class.

drop: stop accessing GdkDrag fields

x11: Stop using GdkDrag fields

wayland: Stop accessing GdkDrag members

Add a GdkDrag::surface property

gdkdrag: remove an unused member

The drag surface is stored in the subclasses and obtained
by via the get_drag_surface vfunc.

wayland: Drop an unused api

Nothing was calling gdk_wayland_drag_set_source_surface.

Add a private struct to GdkDrag

This will force subclasses to use proper api.

Revert "GdkDrop: drop the priv struct"

This reverts commit d927c3bee0f169279d86e13756c5ba4f6cba94e4.

I was wrong. Better to hide the struct fields from subclasses.

dnd: Drop gdk_drag_drop

This is an inconsistently used private api, and just adds
a vfunc indirection for no other benefit.

Merge branch 'wip/matthiasc/kill-non-input-events' into 'master'

Wip/matthiasc/kill non input events

See merge request GNOME/gtk!242

Fix up some doc mishaps

When GdkWindow was renamed to GdkSurface, 'window managers'
turned into 'surface managers'. Thats not cool.

Drop gdk_surface_withdraw

'withdrawn' is an X11-centric concept, and the function
can just as well be replaced by gdk_surface_hide.

gdk: Drop map events

We are really inconsistent about generating these,
and they are not actually used in GTK+. Instead,
add a boolean GdkSurface::mapped property.

win32: Stop using expose and configure events

These no longer exist.

broadway: Stop using configure events

These no longer exist.

gdk: Drop expose events

Replace expose events with a GdkSurface::expose signal.
This is part of the move to use events only for input.

gdk: Drop configure events

Replace configure events with a GdkSurface::size-changed signal.
This is part of the move to use events only for input.

popover: Drop some dead code

This will be done differently, using xdg-popup instead
of subsurfaces, so no point in keeping this.

text display: correct a comment

gl: implement clipping for legacy contexts

gl: fix compilation errors for legacy context shaders

textview: Only create cairo context in gtktextdisplay.c

Everything else is done using GtkSnapshot now, including renaming the
draw_layer vfunc to snapshot_layer.

x11: Set BackPixmap = None unconditionally

Makes it more obvious what we actually want.

Related: #1134

Update POTFILES.in

window: Update opaque region if background-color changes

The opaque region is only set when the background color is opaque. So
we need to do something about it when the background color changes.

However, in the case where a size allocation is going to happen, we
already do this update in size_allocate(), so in that case avoid doing
it twice.

gtk-demo: Update aspect-ratio when selecting new puzzle

textview: Use snapshot on the sidebars

widgetpaintable: Add a hack to make recursion not infloop

Makes the GUADEC talk not crash that I'm supposed to give in 20 minutes.

widgetpaintable: Redo implementation

Instead of instantly invalidating, we now cache the old render node and
do the update in an idle handler.
While that gives us a 1 frame delay, it avoids all the tricky things
like queueing resizes while resizing or queueing draws while drawing.

The only remaining issue (and a *big* one at that) is that a nested
widget paintable will now cause the widget to snapshot its previous
render node when creating a new one. And that one will snapshot its
previous render node, and that one will...
And nothing so far breaks this recursion.

paintable: Fix return_if_fail() statements

I always switch them up...

gl: Don't accidentally use ints for float variables

The int was floor()ing the x/y coordinates of glyphs, which could cause
significant repositioning of glyphs when text was scaled via the MVP.